Skip to content

fix[docs]: add html_static_path config#5037

Open
hwhsu1231 wants to merge 1 commit into
vyperlang:masterfrom
hwhsu1231-fork:add-html-static-path
Open

fix[docs]: add html_static_path config#5037
hwhsu1231 wants to merge 1 commit into
vyperlang:masterfrom
hwhsu1231-fork:add-html-static-path

Conversation

@hwhsu1231

@hwhsu1231 hwhsu1231 commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

What I did

Add the missing config html_static_path in bc5d00a.

If html_static_path = ['_static'] is not specified, any logos placed in the _static subdirectory of the source directory will not be copied to the _static subdirectory under the build directory when building the HTML documentation. This will cause the following two theme option configurations to fail to find the _static/logo-dark|light.svg files:

html_theme_options = {
    "light_logo": "_static/logo-light.svg",
    "dark_logo": "_static/logo-dark.svg",
}

which will result in:

Screenshot_20260606_143701

How I did it

Add the following configuration in conf.py file:

html_static_path = ["_static"]

How to verify it

uv run --group docs make -C docs html

Commit message

fix[docs]: add `html_static_path` config

Add the missing config `html_static_path` in bc5d00a

@hwhsu1231

Copy link
Copy Markdown
Contributor Author

cc: @Sporarum @charles-cooper

Comment thread docs/conf.py Outdated
@Sporarum

Copy link
Copy Markdown
Collaborator

Hello, I have no idea if this is the correct fix, but it looks relatively sane

Add the missing config `html_static_path` in bc5d00a.
@hwhsu1231 hwhsu1231 force-pushed the add-html-static-path branch from 8da5458 to 33706a2 Compare June 11, 2026 08:04
@Sporarum

Copy link
Copy Markdown
Collaborator

Oh please don't force push once the PR is open, it makes it harder to review, everything will get squished anyways

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants